Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: allow multiple configs listed in KUBECONFIG (#779) #781

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

adietish
Copy link
Collaborator

@adietish adietish commented Aug 14, 2024

fixes #779
depends on redhat-developer/intellij-common#237
related to fabric8io/kubernetes-client#6240

This is a tentative, simplified fix where I override the kubernetes-client io.fabric8.kubernetes.client.Config class within the classpath of our plugin.
A proper fix will patch kubernetes-client and then include the fixed client in our plugin. As pointed out in fabric8io/kubernetes-client#6253 (comment) the fixed client is 7.x (we're currently at 6.12) and possibly get backported to 6.13 if the fix is non-intrusive.

@adietish adietish self-assigned this Aug 14, 2024
@adietish adietish changed the title allow multiple configs listed in KUBECONFIG (#779) fix: allow multiple configs listed in KUBECONFIG (#779) Aug 14, 2024
@adietish adietish added this to the 1.5.0 milestone Aug 28, 2024
@adietish adietish force-pushed the issue-779 branch 14 times, most recently from 3068a25 to 8c65c4d Compare September 10, 2024 17:00
@adietish adietish force-pushed the issue-779 branch 6 times, most recently from 9cce0e9 to 0526f10 Compare September 11, 2024 15:28
Copy link

sonarcloud bot commented Oct 1, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot get my clusters to appear
1 participant